auto merge of #235 : rust-lang/cargo/better-test-output, r=alexcrichton

@alexcrichton r?

Fix windows tests

auto merge of #236 : cburgdorf/cargo/kill_lifetimes, r=alexcrichton

This commit removes lifetime annotations
that are now automatically inferred by
the compiler.

kill unnecessary lifetime annotations

This commit removes lifetime annotations
that are now automatically inferred by
the compiler.

Improve test output

Improve error message on failed compile

auto merge of #224 : alexcrichton/cargo/canonical-hash, r=wycats

This should prevent some weird bugs where if you have slightly different sources that you occasionally have to recompile them when cargo switches between one or the other (thinking they're different)

auto merge of #222 : alexcrichton/cargo/no-cargo-toml-in-target, r=wycats

When cargo is building itself, it just ends up getting confused because the test
directory is in this location.

auto merge of #232 : bkoropoff/cargo/fix-repeated-cargo-test, r=alexcrichton

Generate the right target filename for rlibs built in test
mode so that they are correctly moved back out of the old-root
directory.

Add regression test for running `cargo test` twice

Fix failure on second run of 'cargo test'

Generate the right target filename for rlibs built in test
mode so that they are correctly moved back out of the old-root
directory.

auto merge of #231 : alexcrichton/cargo/burningtree, r=burningtree

Factor the metadata into test filenames

Closes #221
Closes #226

fix deprecated usage

Don't look inside target/ for sub-packages

When cargo is building itself, it just ends up getting confused because the test
directory is in this location.

Canonicalize URLs when hashing sources

auto merge of #216 : alexcrichton/cargo/issue-205, r=wycats

As discovered in #205, cargo wasn't properly ensuring a fresh build state for new builds. As commented in #205, this is an easy way to have a non deterministic build which is one of cargo's major goals to avoid. These commits rectify the situation.

All build output is now "obliterated" by moving to a location unknown to the compiler as soon as a build starts. All fresh output is moved back into place automatically, and all dirty output will only have known artifacts available to it.

The major consequence of this strategy is that the location of the output of the `build` command must also change. Instead of spraying into a shared directory, build commands are now contained to their own build directory which is entirely removed on rebuild or entirely preserved on a fresh build. There's more documentation in the new `layout.rs` about the build directory structure.

In general this should not have much impact as a user-facing change.

House all native build output in a per-package dir

In order to ensure there are no stale artifacts as part of a build, this commit
houses all output of native build commands in their own directories. Each
directory is on a per-package basis, and the output is preserved if the package
is fresh or discarded if it is not.

This does not remove the DEPS_DIR environment variable, it just wires it to the
same value as OUT_DIR.

Blow away all directories/files on each build

And aftewards selectively move them back into place if they're fresh. This
prevents stale files from showing up from old builds.

This currently breaks anyone build `build=` scripts, the fix is coming in the
next commit.

Closes #205

auto merge of #220 : erickt/cargo/master, r=alexcrichton

Please don't land this until https://github.com/wycats/hammer.rs/pull/12 is landed, or else it will break the build.

auto merge of #217 : gchp/cargo/docs, r=alexcrichton

There were inconsistencies in the manifest file between using `[[bin]]` and `[[executable]]`. I've changed the examples to use `[[bin]]` for consistency.

Update hammer to fix `cargo build -h`.

auto merge of #212 : fhahn/cargo/version, r=alexcrichton

This PR adds a rustc style version command

auto merge of #219 : alexcrichton/cargo/update-to-mater, r=burningtree

Update to master

Add version command, closes #201

Use [[bin]] instead of [[executable]] in manifest examples

auto merge of #213 : fhahn/cargo/fix-warnings-2, r=alexcrichton

This patch replaces `get()` calls with indexing for vectors and `str::from_utf8_lossy` with `String::from_ut8_lossy` to silence recent rustc warnings

Use indexing instead of get() function to access Vec

auto merge of #208 : alexcrichton/cargo/fix-main-name, r=wycats

The crate name should be the package name, not `main`.

Closes #207

auto merge of #211 : japaric/cargo/update-toml, r=alexcrichton

which fixes the errors caused by Vec.get method deprecation.

Update toml submodule

Fix inferred name of `src/main.rs`

The crate name should be the package name, not `main`.

Closes #207

auto merge of #206 : alexcrichton/cargo/update-toml, r=wycats

TOML recently grew multiline string literals as well as literal strings with no
escaping, which can be used instead of the hokey escape_path() function.

Update toml-rs

TOML recently grew multiline string literals as well as literal strings with no
escaping, which can be used instead of the hokey escape_path() function.

auto merge of #202 : fhahn/cargo/master, r=alexcrichton

Currently it seems like the `DESTDIR` option is ignored during the build process. This poses problems when packaging `rustc`, because it always tries to use `/` as a base path of the prefix. When packaging cargo for Arch Linux, everything should be installed into a separate directory, which content is used to create the package archive. At the moment this step requires `sudo`, which is discouraged in PKGBUILDs. Now, `make install DESTDIR=foo` prepends `DESTDIR` to all relevant paths during installation.

This probably could be archived using `install --prefix=foo/usr/` as well, but the convention to separate prefix and destdir is rather common

Add --destdir option to install.sh

auto merge of #200 : thiagopnts/cargo/master, r=alexcrichton

As I've been looking around cargo's source I ended up making some changes to keep the code style more consistent.

undo inline for better readability

Minor fixes for code style consistency

auto merge of #57 : ebfe/cargo/git-chmod, r=alexcrichton

auto merge of #199 : jdeseno/cargo/cleanup, r=alexcrichton

Touch-up. the return statement here isn't very rusty.

remove unecessary return statement

auto merge of #198 : rust-lang/cargo/another-nightly-fix, r=wycats

I actually verified it this time to make sure it passes!

Re-attempt to fix the nightly windows installer

Remove accidental debugging print

auto merge of #197 : alexcrichton/cargo/fix-windows-dist, r=wycats

The executables all have an `.exe` suffix which needs to be dealt with when moving files around.

Attempt to fix the windows dist bot

auto merge of #196 : alexcrichton/cargo/linker-and-ar, r=wycats

Allow configuratin ar/ld for rustc

This adds a new .cargo/config option which allows configuring the ar and linker
tools that rustc invokes. This should aid in any cross-compilation attempts.

Allow tables in configuration

auto merge of #195 : alexcrichton/cargo/install, r=wycats

This updates the `make install` target and adds a new `make dist` target which
will prepare a distributable tarball with an install script. All work is based
off the equivalent rust nightly tarball and nightly installation scripts.

Closes #159
Closes #184

auto merge of #190 : alexcrichton/cargo/cargo-run, r=wycats

This currently only supports executing the `src/main.rs` convention, no other.

Close #149

auto merge of #189 : alexcrichton/cargo/all-crate-types, r=wycats

While we support the `crate_type` key in the manifest, we need to pass through
all crate types to the `--extern` flag.

Closes #177

Add an installation script to the repo

This updates the `make install` target and adds a new `make dist` target which
will prepare a distributable tarball with an install script. All work is based
off the equivalent rust nightly tarball and nightly installation scripts.

Closes #159
Closes #184

Add a `cargo run` command

This currently only supports executing the `src/main.rs` convention, no other.

Close #149

auto merge of #176 : cburgdorf/cargo/docs_1, r=alexcrichton

I was chasing the `cargo clean` command it's way down to get a better understanding of the inner workings of cargo. Those minor documentation changes and variable renaming are the outcome which I think make the code base just a tiny bit more welcoming :)

Add documentation and make minor variable renames

Pass all formats via --extern for libs

While we support the `crate_type` key in the manifest, we need to pass through
all crate types to the `--extern` flag.

Closes #177

auto merge of #188 : huonw/cargo/remove-src-test, r=wycats

It's not unreasonable to have unittests in a separate submodule of the
crate (being called `test` or `tests`), and having them in their own
file can be very sensible. Thus, the `src/test.rs` implicit default is
likely to trip up some perfectly reasonable use-cases. There's already
the `tests/...` default, so repairing a codebase after this removal is
just moving `src/test.rs` to `tests/whatever_name_you_want.rs`.

Closes #187.

Remove the src/test.rs implicit test.

It's not unreasonable to have unittests in a separate submodule of the
crate (being called `test` or `tests`), and having them in their own
file can be very sensible. Thus, the `src/test.rs` implicit default is
likely to trip up some perfectly reasonable use-cases. There's already
the `tests/...` default, so repairing a codebase after this removal is
just moving `src/test.rs` to `tests/whatever_name_you_want.rs`.

Closes #187.

auto merge of #180 : sfackler/cargo/ndebug, r=wycats

This will disable debug!() and debug_assert!() statements.

Pass --cfg ndebug when debug = false

This will disable debug!() and debug_assert!() statements.

auto merge of #161 : alexcrichton/cargo/cargo-test, r=wycats

This allows `cargo test` usage to filter test being run, use --nocapture, etc.

Pass through extra arguments to `cargo test`

This allows `cargo test` usage to filter test being run, use --nocapture, etc.

auto merge of #172 : huonw/cargo/minor-refactoring, r=alexcrichton

There was some unnecessary duplication and allocation.

Minor refactorings.

There was some unnecessary duplication and allocation.

auto merge of #170 : alexcrichton/cargo/same-name, r=wycats

This allows the dependency queue to properly handle packages with the same
name but from different sources.

A test was added which exercieses this functionality by depending on two
different revs of the same git repo.

auto merge of #163 : o11c/cargo/examples, r=wycats

This add support for `examples/*.rs` being built (as normal bin crates) during `cargo test`,
and `src/test.rs` and `tests/*.rs` being built and run (as test crates) during `cargo test`.

Use PackageId in the DependencyQueue

This allows the dependency queue to properly handle packages with the same
name but from different sources.

A test was added which exercieses this functionality by depending on two
different revs of the same git repo.

Do not run examples during 'cargo test'

Add support for external tests

Add support for examples

auto merge of #169 : cburgdorf/cargo/name_clean, r=alexcrichton

There are other unnamed commands like `verify-project` but I'm unsure if they are **not** listed on purpose because they should be used directly by the user less frequently. However, the clean command seems like a command that should be listed here.

Not sure about the wording though. It could be more abstract like *remove build artifacts* but given it's current behavior I think *remove target directory* is somehow straight forward, too.

list `clean` command in help

auto merge of #167 : alexcrichton/cargo/cross-compile, r=wycats

This also refactored many things along the way, I would recommend reading each commit individually rather than them all as a whole.

Closes #119

Finish plugin support

This commit implements full support for plugins by answering the question of
whether any target needed as a plugin or needed as a target dependency. This
commit builds on the previous abstractions to enable parallel compilation
wherever possible.

auto merge of #166 : nielsle/cargo/command-env, r=alexcrichton

Quickfix for the build after https://github.com/rust-lang/rust/pull/15353

Use command.env_set_all() instead of command.env()

Fixes build after
https://github.com/rust-lang/rust/pull/15353

Add some simple tests for cross compilation

Refactor the parallel job queue to its own module

Refactor fingerprint logic out of cargo_rustc

Refactor Context into its own submodule

Refactor `Job` to its own module

Improve intra-target parallelism

With cross compilation soon on the horizon, it may be required to build a
library for both the host and target architectures. These two copies can
certainly be built in parallel. Additionally, all binaries produced by a package
can also be built in parallel, but are currently forced to be built
sequentially.

This commit improves this parallelism by allowing each job to create more work
before the package is considered built. Only after all targets have been built
is the new fingerprint written.

Update to master

Implement a `plugin` flag for targets

If `plugin = true` is specified, then the target is considered a compiler plugin
which implies two separate properties:

* The library must be compiled as a dylib
* The library must be compiled for the host architecture

Add bare-bones cross compilation support to cargo

This adds a new flag, --target, to the `cargo-build` command. This flag will
indicate that all output should be scoped under the `$triple` directory inside
of the output `target` directory. The compiler is invoked with `--target
$triple` and all custom build commands are passed `TRIPLE` if one is provided.

auto merge of #162 : rust-lang/cargo/fix-travis, r=tinyfix

Fix travis builds

auto merge of #157 : rust-lang/cargo/graph-resolve, r=alexcrichton

This incorporates previous bugfixes by @alexcrichton that are still stuck in bors.

Fix some tests

Tighten up some tests

Fix filtering for test deps

This fixes `cargo test` to only test the *local* package

No need for a TempDir, use `-` as input

impl Show for Profile

Add -u to `cargo test`

More updates to master

Make README nightlies links clickable

ToStr::to_str -> ToString::to_string

Conflicts:
src/cargo/core/resolver.rs
src/cargo/ops/cargo_rustc.rs